Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing module name #68

Merged
merged 1 commit into from
Sep 7, 2023
Merged

Add missing module name #68

merged 1 commit into from
Sep 7, 2023

Conversation

rajbos
Copy link
Collaborator

@rajbos rajbos commented Sep 7, 2023

What are the incoming changes?

πŸ€– Generated by Copilot at fd03df2

Refactored entrypoint.ps1 to use a variable for the module name. This makes the code easier to read and modify.
​

πŸ€– Generated by Copilot at fd03df2

Module variable
Replaces hard-coded string
Code is more graceful

Why are they needed

How has this been achieved?

πŸ€– Generated by Copilot at fd03df2

  • Extract module name "powershell-yaml" to a variable $moduleName for reuse and consistency (link, link)
  • Import the module $moduleName using the -MinimumVersion parameter to ensure compatibility (link)

@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2023

Dependency Review

βœ… No vulnerabilities or license issues found.

Scanned Manifest Files

@rajbos rajbos merged commit 52f280c into main Sep 7, 2023
11 checks passed
@rajbos rajbos deleted the rajbos-patch-2 branch September 7, 2023 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant